Skip to content

[OSDOCS-20358]: CQA: Automated DR with OADP for HCP (part 3 of 4)#114224

Open
lahinson wants to merge 1 commit into
openshift:mainfrom
lahinson:osdocs-20358-cqa-hcp-oadp-auto-3
Open

[OSDOCS-20358]: CQA: Automated DR with OADP for HCP (part 3 of 4)#114224
lahinson wants to merge 1 commit into
openshift:mainfrom
lahinson:osdocs-20358-cqa-hcp-oadp-auto-3

Conversation

@lahinson

@lahinson lahinson commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

@lahinson lahinson added this to the Continuous Release milestone Jun 26, 2026
@openshift-ci openshift-ci Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 26, 2026
@lahinson lahinson added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jun 26, 2026
@ocpdocs-previewbot

ocpdocs-previewbot commented Jun 26, 2026

Copy link
Copy Markdown

@lahinson lahinson changed the title [OSDOCS-20358]: CQA: Automated DR with OADP for HCP (part 3 of 3) [OSDOCS-20358]: CQA: Automated DR with OADP for HCP (part 3 of 4) Jun 26, 2026
@lahinson lahinson force-pushed the osdocs-20358-cqa-hcp-oadp-auto-3 branch from bb414c7 to 21a4461 Compare July 1, 2026 19:20
@lahinson lahinson added the merge-review-needed Signifies that the merge review team needs to review this PR label Jul 1, 2026
@mburke5678 mburke5678 added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Jul 1, 2026
<7> Specifies that the `fs-backup` backing up method for persistent volumes (PVs) is not used.
+
* `metadata.name` specifies the name for your `Backup` resource.
* `spec.includedNamespaces` specifies namespaces to back up objects from. You must include your hosted cluster namespace and the hosted control plane namespace. Replace `<hosted_cluster_namespace>` with the name of the hosted cluster namespace. Replace `<hosted_control_plane_namespace>` with the name of the hosted control plane namespace.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe tighten this up a bit?

Suggested change
* `spec.includedNamespaces` specifies namespaces to back up objects from. You must include your hosted cluster namespace and the hosted control plane namespace. Replace `<hosted_cluster_namespace>` with the name of the hosted cluster namespace. Replace `<hosted_control_plane_namespace>` with the name of the hosted control plane namespace.
* `spec.includedNamespaces` specifies namespaces to back up objects from. You must replace `<hosted_cluster_namespace>` with the name of the hosted cluster namespace and replace `<hosted_control_plane_namespace>` with the name of the hosted control plane namespace.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good suggestion. Will do.

@mburke5678

Copy link
Copy Markdown
Contributor

@lahinson One suggestion. I don't fee comfortable merging or adding the ok-to-merge label until the previews are working again.

@mburke5678 mburke5678 removed merge-review-in-progress Signifies that the merge review team is reviewing this PR merge-review-needed Signifies that the merge review team needs to review this PR labels Jul 1, 2026
@lahinson lahinson force-pushed the osdocs-20358-cqa-hcp-oadp-auto-3 branch from 21a4461 to 767a686 Compare July 1, 2026 20:32
@openshift-ci

openshift-ci Bot commented Jul 1, 2026

Copy link
Copy Markdown

@lahinson: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.20 branch/enterprise-4.21 branch/enterprise-4.22 branch/enterprise-5.0 do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants